Skip to content

[pull] main from nodejs:main#104

Merged
pull[bot] merged 4 commits into
zys-contrib:mainfrom
nodejs:main
Jul 25, 2025
Merged

[pull] main from nodejs:main#104
pull[bot] merged 4 commits into
zys-contrib:mainfrom
nodejs:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Jul 25, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

RafaelGSS and others added 4 commits July 25, 2025 13:22
This option will measure how long the benchmark
takes to run. Example:

```
$ ./node benchmark/run.js --track assert

assert/assertion-error.js
assert/assertion-error.js size=2 n=200: 22,987.24012781365
assert/assertion-error.js size=75 n=200: 296.2362823364434
[740ms] assert/assertion-error.js

...
```

Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: #59174
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
The timeout is unnecessary since we are testing for certificate
failure. It can cause flakes on very slow machines where the
request cannot be finished in 1 second.

PR-URL: #59165
Fixes: #59166
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #59160
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
PR-URL: #59059
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@pull pull Bot locked and limited conversation to collaborators Jul 25, 2025
@pull pull Bot added the ⤵️ pull label Jul 25, 2025
@pull pull Bot merged commit 345a550 into zys-contrib:main Jul 25, 2025
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants